Skip to content

[MOSIP-23551] Added validators for doctype and docCat Fileds in Valid document#1180

Open
balaji-alluru wants to merge 4 commits intomosip:developfrom
balaji-alluru:develop-patch-doc_valid
Open

[MOSIP-23551] Added validators for doctype and docCat Fileds in Valid document#1180
balaji-alluru wants to merge 4 commits intomosip:developfrom
balaji-alluru:develop-patch-doc_valid

Conversation

@balaji-alluru
Copy link
Contributor

No description provided.

…cument entity

Signed-off-by: Balaji <74903654+balaji-alluru@users.noreply.github.com>
Signed-off-by: Balaji <74903654+balaji-alluru@users.noreply.github.com>
Signed-off-by: Balaji <74903654+balaji-alluru@users.noreply.github.com>
Signed-off-by: Balaji <74903654+balaji-alluru@users.noreply.github.com>
Comment on lines +5 to +6
public static final String INVALID_DOC_CAT_CODE = "DocCategory is Invalid";
public static final String INVALID_DOC_TYPE_CODE ="DocType is Invalid";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be error message or error code?

return properties;
}

@Bean("DocCatCodes")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let the bean name start with lowercase

return getDocCatCodes();
}

@Bean("DocTypeCodes")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let the bean name start with lowercase

mosip.kernel.config.server.file.storage.uri=https://qa3.mosip.net/config/print/mz/qa3-1.1.5/
mosip.admin.identityMappingJson=identity-mapping.json
mosip.admin.applicant-details.exposed-identity-fields=dob,applicantPhoto
mosip.admin.doccodes-cleanup.fixed-rate=24
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a comment to specify the value unit? and also what codes are cleaned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants